New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: change the concurrency queue locking mechanism, example readme #1

Merged

MichaelGHSeg merged 2 commits into main from oscb/concurrencyFix

Jun 28, 2023

+115 −13

Contributor

oscb commented May 9, 2023

The way that timeline queued the processing was prone to concurrency issues depending on where the execution halted which could lead to deadlocks or accessing wrong indexes.
Reverting to a simpler locking queue mechanism that guarantees that the operations will always be executed in line and the futures completed without swapping values
Added instructions on how to run the example app for the first time.


          fix: change the concurrency queue locking mechanism, example readme

9e2e7cf

bsneed approved these changes

View reviewed changes

bsneed left a comment

I love how much simpler it seems to be.


          chore: add comments

21519da

marketoptiontend linked an issue

that may be closed by this pull request

Closed

MichaelGHSeg merged commit 505ac1e into main

MichaelGHSeg deleted the oscb/concurrencyFix branch

June 28, 2023 15:05

t-unit mentioned this pull request

ScreenObserver not working with nested navigation #30

Open

GeorgePagMezzi mentioned this pull request

Appsflyer help #99

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet